menu: Remove have_position flag

It's always FALSE.

menu: Remove an unnecessary gtk_widget_destroy() call

The unref will already properly free the menu item's resources.

Don't fail the build if c++ isn't found

The intention of this check was to skip the keyword
test if no c++ compiler is found. But the meson
docs say that add_languages() will abort unless we
pass required: false.

Merge branch 'fix-shell-colorpicker' into 'master'

colorpickershell: Unpack the tuple returned from PickColor()

See merge request GNOME/gtk!296

print portal: don't leak window handles

Pointed out by Christian Persch.

Closes https://gitlab.gnome.org/GNOME/gtk/issues/1274

print portal: Plug a memory leak

Pointed out by Christian Persch.

Closes https://gitlab.gnome.org/GNOME/gtk/issues/1271

Use defines for bus names

This helps eradicate typos (we had some here).

Closes https://gitlab.gnome.org/GNOME/gtk/issues/1267

Merge branch 'sort' into 'master'

updateiconcache: Sort list of entries

See merge request GNOME/gtk!297

font chooser: Remove an unneeded ifdef

We can just always create the axes hashtable.

updateiconcache: Sort list of entries

because filesystem readdir order is indeterministic.

Without this patch, building openSUSE's balsa package
had variations between builds in /usr/share/balsa/icon-theme.cache

Merge branch 'master' into 'master'

window: Fix memory leak

See merge request GNOME/gtk!295

colorpickershell: Unpack the tuple returned from PickColor()

When calling PickColor on org.gnome.Shell, we get back an "a{sv}", which
GDBus provides to us as "(a{sv})".

At the minute we're not unpacking this tuple, and so picking fails with
messages like:

  GLib-CRITICAL **: 13:38:19.439: g_variant_lookup_value: assertion 'g_variant_is_of_type (dictionary, G_VARIANT_TYPE ("a{s*}")) || g_variant_is_of_type (dictionary, G_VARIANT_TYPE ("a{o*}"))' failed

  Gtk-WARNING **: 13:38:19.439: Picking color failed: No color received

Let's unpack it.

window: Fix memory leak

https://gitlab.gnome.org/GNOME/gtk/issues/1268

Update Turkish translation

Update Polish translation

Merge branch 'gtk-4-improve-error-message' into 'master'

gtkplacessidebar: Improve error message when unlocking volume fails

See merge request GNOME/gtk!267

Update Polish translation

migration guide: Mention GtkEventBox

Its gone.

Closes: https://gitlab.gnome.org/GNOME/gtk/issues/1243

README: Fix Gdk doc website link

README: Document nightly docs location

Useful for exposing the documentation early.

gtkplacessidebar: Improve error message when unlocking volume fails

Updated Lithuanian translation

widget: Rename width and height member to {width,height}_request

Because that's much closer to the meaning of those values.

main: Pass toplevel_widget directly to GtkWidget API

We already have the toplevel as GtkWidget: toplevel_widget.

sizerequestcache: Inline out-var assignments

The additional assignment to the old result variable just adds an
indirection even though we know the point where we assign it in all
cases. Just pass the values out and return in those cases instead.

widgetprivate: Remove useless prototype

We implement that function lower down in that header file.

pathbar: Remove priv pointer

menuitem: Don't redraw unnecessarily

The gtk_widget_(un)set_state_flags calls before will do this if
necessary.

menu: Use correct content height

menu: Allocate children according to scroll_offset

This was done by simply moving the bin_window before but now we have to
do it ourselves.

levelbar: Don't return a double value from a gboolean function

acitonbar: Remove snapshot implementation

modelbutton: Remove snapshot implementation

Previously, GtkBin was only snapshot'ing its one and only child, but
nowadays it doesn't implement snapshot at all and the default
implementation in GtkWidget just snapshots all child widgets, which is
exactly what the implementation in gtkmodelbutton.c was doing.

Make color picker circular

The need for center alignment was pointed out by Timm Baeder.

Update German translation

Merge branch 'sidebar-spinner-visibility' into 'master'

sidebarrow: Hide busy_spinner by default

See merge request GNOME/gtk!285

sidebarrow: Hide busy_spinner by default

Since the original implementation was likely based on GTK+ 3, the change
in default visibility might have not been considered, which results in
all rows suddenly sporting a visible spinner when opening a fresh file
chooser.

Merge branch 'fix-app-id' into 'master'

Make xdg-shell's app_id match GApplication's application-id property

Closes #653

See merge request GNOME/gtk!284

color picker: Also try kwin

use a kwin color picker when we run under kwin.

Only use the portal color picker when appropriate

This is just moving this check around.

Add a kwin color picker

This uses the org.kde.kwin.ColorPicker interface.

Merge branch 'gbsneto/issue-1258' into 'master'

listbox: Unparent child after removing from sequence

Closes #1258

See merge request GNOME/gtk!283

listbox: Store child iter in a variable when removing

Unparenting a GtkListBoxRow can drop its last reference, which
will free its memory. Right after unparenting, though, we were
accessing the row's iter - which assumes that the row is still
alive. This causes a crash when, for example, binding two or
more models to the listbox.

Fix that by storing the iter in a variable, and not trying to
access it after unparenting. After unparenting, the variables
that are potentially garbage were explicitly assigned NULL for
clarity.

Fixes https://gitlab.gnome.org/GNOME/gtk/issues/1258

demos: Make desktop file names match the application id

examples: Use appliction id as desktop file base name

docs: Explain desktop file id == application_id in migration guide

gdk: Use application_id as xdg_shell's app_id if available

This (manually) reverts commit e1fd87728dd841cf1d71025983107765e395b152

This makes sure xdg_shell's app_id matches the DBus name of the the
appliation when using GtkApplication. It also allows the compositor to
derive the desktop file ID from the app_id:

   https://cgit.freedesktop.org/wayland/wayland-protocols/tree/stable/xdg-shell/xdg-shell.xml?id=298d888ac718eae57ff2245d373d4327074506ea#n598

Closes #653

container: Remove focus chains

They're prectically unused.

Widgets who want to influcence the focus chain behavior should implement
GtkWidgetClass::focus and deal with focus there.

widgetfocus: Fix copy/paste error

This lead to erratic focus behavior with <Tab> in hboxes, where the
smallest child got focus first instead of the leftmost child.

colorpicker: Fix gcc warning

Mention gtk_widget_show_all in the migration guide

This function is gone.

Closes https://gitlab.gnome.org/GNOME/gtk/issues/1242

Merge branch 'issue1214-tooltips-win32-gtk4' into 'master'

Gdk-Win32: Correct handling of transient state changes

See merge request GNOME/gtk!280

Gdk-Win32: Correct handling of transient state changes

See merge request !248

bindings: Remove GTK_TYPE_IDENTIFIER

bindings now treat identifiers and strings the same way.

The only difference was that one allowed lookup of enum/flags by name
while the other didn't and g_warning()ed. Now both work.

color picker: Better debug spew

This will help in debugging issues.

Updated Czech translation

gdk: Fix Windows and OS X backend for keyevent.string removal

treeview: Fix compiler warning

gdk: Remove key_event->string and key_event->length

They're unused.

gdk: Remove gdk_event_get_string()

You want to use an IM module to get strings out of keypresses, not some
crude hack that only works on X11 and Wayland anyway.

Update POTFILES.skip

Updated Czech translation

colorpickerportal: Fix memory leak

inspector: Remove unnecessary cast

Pointed out by Timm.

Merge branch 'gtk-4-add-spinner-to-sidebar-row' into 'master'

gtkplacessidebar: Show busy spinner in sidebar row during mount ops

See merge request GNOME/gtk!261

Merge branch 'wip/carlosg/event-final-cleanup' into 'master'

Wip/carlosg/event final cleanup

See merge request GNOME/gtk!279

tests: Port testwidgetfocus to using a motion controller

demos: Update changedisplay demo to using GtkGesture

tests: Port motion-compression to using a motion controller

gtkscrolledwindow: Use controller for motion capturing

Perform scrollbar visibility checks through a motion controller,
always based on GtkScrolledView-relative coordinates. The captured
event handler remains though, for a tiny bit of GDK_SCROLL event
handling.

gtkwindow: Replace captured event handler with motion controller

testsuite: Update window test to use gestures when interactive

tests: Use gesture in testtreepos

tests: Use gesture to popup menu in testiconview

tests: Update testgrid to use gestures

demos: Update hypertext demo to use controllers

demos: Update search_entry2 demo to use key capturing API

No more GdkEvent juggling.

demos: drop "Event axes" demo

It's too lowlevel to be practical anymore with gtk4, and there's
the Paint and Gestures demos to demonstrate input capabilities
better than this.

gtk-demo: Add pad controller handling to "Paint" demo

So it's more lifelike.

gtktreeview: Use key controller on search entry handling

Use a distinct key controller so we correctly handle navigation
across matches and search cancellation. As the events are forwarded
to the search_window, those need to be pushed down the entry manually.

gtktreeview: Replace event vmethod with key controller

gtkcombobox: Use keycontroller on GtkCellEditable implementation

gtkentry: Use keycontroller on GtkCellEditable implementation

inspector: Use gestures/controllers for inspect button

gtkcombobox: Use controller to handle combobox keybindings in menus

Instead of an ::event callback.

Merge branch 'fix-gs-issue-404' into 'master'

window: Fallback to CSD titlebar in focus-chain

Closes gnome-software#404

See merge request GNOME/gtk!271

Merge branch 'devel-styling' into 'master'

Devel styling

See merge request GNOME/gtk!168

Adwaita: devel style

- nightly/development versions of apps should get a styled
  headerbar.

https://gitlab.gnome.org/GNOME/gtk/merge_requests/142

window: Fallback to CSD titlebar in focus-chain

CSD titlebar are included in the focus-chain. The logic used makes sure that the
initial focus avoids the titlebar, but tabbing around will eventually get there.
This logic fails in case the window has no other focusable widgets apart from
the ones in the header-bar. If this happens keynav focus will be lost. To handle
the above scenario, we need to fallback to focus the header-bar (if any).

Fixes: https://gitlab.gnome.org/GNOME/gnome-software/issues/404

treeview: Get rid of child surfaces

Drop the drag-highlight and drag surfaces. The highlighting
is broken anyway, so just drop it for now. And for dragging
the header button, we can just position it properly, that
works just as well as this reparenting approach.

Update Polish translation

Migration guide: mention size-allocate

Document the new argument here.

Closes: https://gitlab.gnome.org/GNOME/gtk/issues/1245

Remove an outdated comment

We no longer have semi-private headers.

vulkan: fix warning from g_clear_pointer() changes

imwayland: fix potential leak of attr list

This fixes a potential leak of a PangoAttrList that is set when chaining
up to the parent get_preedit_string(). We check to see if the attr list
was created and reuse it instead of leaking the previous value.

Updated Lithuanian translation

Merge branch 'wip/gtk-menu-popup-no-more' into 'master'

Remove legacy GtkMenu popup APIs

See merge request GNOME/gtk!270

Merge branch 'color-picker' into 'master'

Color picker

See merge request GNOME/gtk!273

tests: Update a11y tests

Some things changed.

Add a color picker implementation for gnome-shell

This adds a GtkColorPicker implementation that talks
to gnome-shell to get a color.